Skip to content

Use super parameters in more places, linted by the analyzer#9815

Merged
srawlins merged 2 commits intoflutter:masterfrom
srawlins:use-super
Apr 30, 2026
Merged

Use super parameters in more places, linted by the analyzer#9815
srawlins merged 2 commits intoflutter:masterfrom
srawlins:use-super

Conversation

@srawlins
Copy link
Copy Markdown
Contributor

This fixes a breakage in CI: https://github.com/flutter/devtools/actions/runs/25175731644/job/73807795671?pr=9814

A few parameters are also just removed, as they became seen as "unused," and no calls to the constructor passes a value for the parameter.

@srawlins srawlins requested a review from a team as a code owner April 30, 2026 20:19
@srawlins srawlins requested review from elliette and removed request for a team April 30, 2026 20:19
@srawlins srawlins requested a review from kenzieschmoll April 30, 2026 20:20
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request refactors the _TestScreen constructor across several test files to use Dart's super-initializer parameters, improving conciseness. A concern was raised regarding the showFloatingDebuggerControls parameter in scaffold_debugging_controls_test.dart, which is currently unused and should be removed for consistency with the other refactored files.

@srawlins srawlins merged commit a14ede6 into flutter:master Apr 30, 2026
129 of 132 checks passed
@srawlins srawlins deleted the use-super branch April 30, 2026 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants